Skip to content

Add interactive axioms notebook#16

Closed
Haleshot wants to merge 2 commits intomainfrom
haleshot-002
Closed

Add interactive axioms notebook#16
Haleshot wants to merge 2 commits intomainfrom
haleshot-002

Conversation

@Haleshot
Copy link
Copy Markdown
Contributor

@Haleshot Haleshot commented Feb 3, 2025

📝 Summary

Add Intro to axoims notebooks; showcasing rules of probability in an interactive way.

TODO:

  • Fix last callout not rendering (the check answer task).

📋 Checklist

  • I have included package dependencies in the notebook file using --sandbox
  • If adding a course, include a README.md
  • Keep language direct and simple.

@Haleshot Haleshot requested a review from akshayka February 3, 2025 11:48
@Haleshot Haleshot self-assigned this Feb 3, 2025
Copy link
Copy Markdown
Contributor

@akshayka akshayka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the attempt to make everything interactive and fun! But in this case, the concept of probabilities summing to 1 is simple enough that I think most of the interactive examples detract from the learning instead of adding. I think having just one interactive example for this notebook, the first one with a scale, is sufficient.

@Haleshot
Copy link
Copy Markdown
Contributor Author

Haleshot commented Feb 4, 2025

I like the attempt to make everything interactive and fun! But in this case, the concept of probabilities summing to 1 is simple enough that I think most of the interactive examples detract from the learning instead of adding. I think having just one interactive example for this notebook, the first one with a scale, is sufficient.

Made some changes accordingly. Current notebook explains the axioms with appropriate examples. Kept the first interactive example at the start (with the scale) the same.

Hope it's good now.

@Haleshot Haleshot closed this Feb 11, 2025
@Haleshot Haleshot reopened this Feb 11, 2025
@Haleshot
Copy link
Copy Markdown
Contributor Author

Closing this PR now; new one up here: #31

@Haleshot Haleshot closed this Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants